Integrate Intercom SDK for helpdesk functionality #675
Conversation
WalkthroughReplaced Freshworks support widget with the Intercom Messenger SDK: added the dependency, initialized Intercom in AppHead, and removed Freshworks prefill code and exported hook from SomethingWentWrong. No other public APIs were added. Changes
Sequence Diagram(s)sequenceDiagram
participant Browser
participant AppHead
participant Intercom
Browser->>AppHead: Load app
AppHead->>Intercom: initialize(appId)
Note right of Intercom: Intercom messenger loaded
Browser->>SomethingWentWrong: Error occurs
SomethingWentWrong->>Intercom: showNewMessage(error details)
Intercom->>Browser: Open messenger with message
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
💤 Files with no reviewable changes (1)
✨ Finishing Touches
🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
🔭 Outside diff range comments (1)
web-server/src/components/AppHead.tsx (1)
13-32: Potential issue: Intercom may initialize multiple times.The Intercom initialization is inside a useEffect with dependencies that may cause it to re-run when the router changes. This could lead to multiple initializations of the Intercom widget.
Consider moving the Intercom initialization to a separate useEffect with an empty dependency array:
useEffect(() => { const isDev = process.env.NEXT_PUBLIC_APP_ENVIRONMENT === 'development'; - Intercom({ - app_id: 'kckm1m2e' - }); + }, [router.asPath, router.events, router.isReady, router.pathname]); + + useEffect(() => { + Intercom({ + app_id: process.env.NEXT_PUBLIC_INTERCOM_APP_ID || 'kckm1m2e' + }); + }, []); + + useEffect(() => { + const isDev = process.env.NEXT_PUBLIC_APP_ENVIRONMENT === 'development'; if (!isDev) { const onFocus = () => track('WINDOW_FOCUS'); const onBlur = () => track('WINDOW_BLUR'); const onUnload = () => track('WINDOW_UNLOAD'); window.addEventListener('focus', onFocus); window.addEventListener('blur', onBlur); window.addEventListener('beforeunload', onUnload); return () => { window.removeEventListener('focus', onFocus); window.removeEventListener('blur', onBlur); window.removeEventListener('beforeunload', onUnload); }; } }, [router.asPath, router.events, router.isReady, router.pathname]);
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
web-server/yarn.lockis excluded by!**/yarn.lock,!**/*.lock
📒 Files selected for processing (3)
web-server/package.json(1 hunks)web-server/src/components/AppHead.tsx(2 hunks)web-server/src/components/SomethingWentWrong/SomethingWentWrong.tsx(2 hunks)
⏰ Context from checks skipped due to timeout of 90000ms (1)
- GitHub Check: All file linting
🔇 Additional comments (2)
web-server/package.json (1)
18-18: Confirm @intercom/messenger-js-sdk version and beta statusThe dependency in web-server/package.json (line 18):
"@intercom/messenger-js-sdk": "^0.0.14",is indeed the latest published release—but it’s still classified as a beta (0.0.x) with no non-beta version available.
Before proceeding, please ensure you’re comfortable with the implications of a beta SDK:
- Pin the dependency to an exact version (e.g.
"0.0.14") if you need to avoid unintended minor bumps.- Add integration or end-to-end tests around any Intercom interactions to catch breaking changes early.
- Monitor the package’s changelog or GitHub issues for upcoming API changes or security advisories.
web-server/src/components/SomethingWentWrong/SomethingWentWrong.tsx (1)
1-1: Good implementation approach for replacing Freshworks with Intercom.The import and usage of
showNewMessagealigns well with the overall migration strategy.
web-server/src/components/SomethingWentWrong/SomethingWentWrong.tsx
Outdated
Show resolved
Hide resolved
|
|
2025-07-16.18-59-40.mp4
Integrate the Intercom SDK to enhance helpdesk prefill functionality, replacing the previous Freshworks integration with the new messaging feature.
Summary by CodeRabbit